Add Missing Tools column to logs overview table with verbose mode support#1319
Merged
Add Missing Tools column to logs overview table with verbose mode support#1319
Conversation
- Add MissingToolCount field to WorkflowRun struct - Populate count from MissingTools array in ProcessedRun - Display "Missing" column in overview table alongside Errors/Warnings - Include total missing tools count in summary row - Add comprehensive test coverage for missing tools display Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Add logging for missing tools in runs
Add Missing Tools column to logs overview table
Oct 7, 2025
pelikhan
approved these changes
Oct 7, 2025
Contributor
|
@copilot in verbose mode, show full list of missing tools |
- Pass verbose parameter to displayLogsOverview - In verbose mode, display actual tool names instead of count - Update function signature to accept ProcessedRun for access to tool details - Truncate long tool lists to fit table (max 30 chars) - Update all tests to use new signature with ProcessedRun - Test both verbose and non-verbose modes Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Contributor
Author
Done! The overview table now shows full tool names in verbose mode. Normal mode:
Verbose mode (
Commit: Added in commit that enhances the overview table to display actual tool names when |
Copilot
AI
changed the title
Add Missing Tools column to logs overview table
Add Missing Tools column to logs overview table with verbose mode support
Oct 7, 2025
Contributor
|
@copilot format, lint, fix tests |
- Remove unused containsString helper function - Remove unused strings import - All linter checks now pass Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Contributor
Author
Done! Fixed all linting issues. Changes:
Validation:
Commit: Fixed in latest commit. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add Missing Tools Column to Logs Overview Table ✅
Problem Solved
The
logscommand now prominently displays missing tools in the main overview table. Previously, missing tools were parsed correctly but only shown in a separate detailed summary section, making them easy to overlook.Solution
Added a "Missing" column to the logs overview table showing the count of missing tools for each workflow run, positioned alongside the existing Errors and Warnings columns for immediate visibility.
Enhanced with verbose mode support: When running with the
--verboseflag, the Missing column displays the actual tool names instead of just the count.Changes Made
MissingToolCountfield toWorkflowRunstructMissingToolsarray when preparing overview displaydisplayLogsOverviewto acceptProcessedRunandverboseparametersmake fmtVisual Example
Normal Mode (
gh aw logs):Verbose Mode (
gh aw logs --verbose):Note: In verbose mode, the "Missing" column shows actual tool names (truncated to 30 chars if needed).
Test Coverage
Created
pkg/cli/logs_overview_test.gowith 10 comprehensive tests:Implementation Details
Modified Files:
pkg/cli/logs.go- Enhanced to support verbose modeMissingToolCount intfield to WorkflowRun structdisplayLogsOverviewsignature to acceptProcessedRunandverboseboolpkg/cli/logs_overview_test.go- Updated tests to use new signatureKey Benefits
--verboseBackward Compatibility
✅ No breaking changes - all existing functionality preserved
✅ Additional column enhances existing display
✅ Detailed missing tools summary section still displays below the overview
Testing Results
How It Works
extractMissingToolsFromRun()parsesagent_output.jsonartifactsProcessedRun.MissingToolsarraylen(pr.MissingTools)WorkflowRun.MissingToolCountfielddisplayLogsOverview()receivesverboseparameterdisplayMissingToolsAnalysis()still shows detailed breakdown separatelyThis implementation ensures that missing tools are now prominently visible in the main logs overview, with optional verbose mode to see the actual tool names.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.